Skip to content

Add spare Vulkan RT scene slot for pipelining - #156

Merged
anirul merged 3 commits into
masterfrom
codex/fix-vulkan-rt-pipeline-slot-count
Apr 9, 2026
Merged

Add spare Vulkan RT scene slot for pipelining#156
anirul merged 3 commits into
masterfrom
codex/fix-vulkan-rt-pipeline-slot-count

Conversation

@anirul

@anirul anirul commented Apr 9, 2026

Copy link
Copy Markdown
Owner

Summary

  • add a spare Vulkan hardware RT scene slot beyond the two in-flight frame slots
  • avoid reusing an RT scene slot while older in-flight frames may still reference it
  • remove the default active-slot initialization so the active scene is only set once a real RT scene has been built

Why

The pipelined Vulkan RT path previously allocated only kMaxFramesInFlight scene slots. With two frames in flight, that left no spare slot to build the next TLAS while both current frames were still using previous scene snapshots. That can produce alternating-frame corruption and invalid lighting, which matched the observed white/incorrect every-other-frame behavior.

Validation

  • cmake --build C:/Users/frede/github/Frame/build/windows --config Debug --target FrameVulkan FrameVulkanTest
  • 02_Dragon.exe --device=vulkan --auto_exit_seconds=3

Notes

@anirul
anirul merged commit ad8658a into master Apr 9, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant